| commit | 1622a02855f769d64d3d286eaa70a8640532310a | [log] [tgz] |
|---|---|---|
| author | Philipp Hancke <philipp.hancke@googlemail.com> | Mon Jun 11 10:00:53 2018 |
| committer | Philip Jägenstedt <philip@foolip.org> | Mon Jun 11 13:28:44 2018 |
| tree | 25d9d11926beb3b07c10df6f52d64eae69652306 | |
| parent | 044cb1b9a719fd2ff423e0bcd2e62dd6e547d52d [diff] [blame] |
webrtc wpt: add cleanup to close peerconnections adds a pc.close() to all RTCPeerConnections in promise_test and async_test. This is codemod-powered by this: https://github.com/fippo/webrtc-codemods/blob/e844b2467cbb76a231c113366b2451cf248f53bc/wpt-rtcpeerconnection-no-helpers BUG=836871 Change-Id: Iee84ef938bf477a55612b6012c6de464c6aea55b Reviewed-on: https://chromium-review.googlesource.com/1049983 Commit-Queue: Henrik Boström <hbos@chromium.org> Reviewed-by: Henrik Boström <hbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#565591}
diff --git a/webrtc/RTCPeerConnection-connectionState.html b/webrtc/RTCPeerConnection-connectionState.html index 1f2f1b7..d8e9411 100644 --- a/webrtc/RTCPeerConnection-connectionState.html +++ b/webrtc/RTCPeerConnection-connectionState.html
@@ -107,8 +107,11 @@ */ async_test(t => { const pc1 = new RTCPeerConnection(); + t.add_cleanup(() => pc1.close()); const pc2 = new RTCPeerConnection(); + t.add_cleanup(() => pc2.close()); + const onConnectionStateChange = t.step_func(() => { const { connectionState } = pc1; if(connectionState === 'connected') { @@ -136,7 +139,6 @@ exchangeIceCandidates(pc1, pc2); doSignalingHandshake(pc1, pc2); - }, 'connection with one data channel should eventually have connected connection state'); /*